drivers: power: lt3074: add driver for LT3074 #2408
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The LT3074 is a low voltage, ultralow noise, and ultrafast transient response linear regulator with a PMBus serial interface. The device supplies up to 3A with a typical dropout voltage of 45mV. Signed-off-by: Cedric Encarnacion <[email protected]>
This adds tinyIIO support for the LT3074 device. This exposes output voltage and current, temperature, and bias voltage channels. Signed-off-by: Cedric Encarnacion <[email protected]>
cencarna
requested review from
buha,
CiprianRegus,
dbogdan,
amiclaus,
rbolboac and
danmois
as code owners
Add initial project files for both basic and IIO examples for LT3074. Signed-off-by: Cedric Encarnacion <[email protected]>
Add README.rst documentation file for LT3074 alongside other documentation related files. Signed-off-by: Cedric Encarnacion <[email protected]>
Add README.rst documentation file for project alongside other documentation related files. Signed-off-by: Cedric Encarnacion <[email protected]>
cencarna
force-pushed
the
staging/lt3074
branch
from
43aab10 to
6a215c2
Compare
amiclaus
approved these changes
Jan 13, 2025
amiclaus
reviewed
Jan 13, 2025
| * @return ret - Result of the reading procedure. | ||
| * In case of success, the size of the read data is returned. | ||
| */ | ||
| static int32_t lt3074_iio_reg_read(void *dev, uint32_t reg, uint32_t *readval) |
There was a problem hiding this comment.
actually there is one comment: the return type of the functions should be int.
There was a problem hiding this comment.
These are mapped to .debug_reg_read and .debug_reg_write which return int32_t. Compiler also gives warnings about incompatible pointers. Should I still proceed with int?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Description
The LT3074 is a low voltage, ultralow noise, and ultrafast transient
response linear regulator with a PMBus serial interface. The device
supplies up to 3A with a typical dropout voltage of 45mV.
This pull request adds the no-OS and IIO driver, as well as examples
and documentations, to quickly evaluate the LT3074.
PR Type
PR Checklist